Skip to content

[O2B-1461] Export refactor#1947

Merged
xsalonx merged 35 commits into
mainfrom
xsalonx/export/O2B-1293/export
Aug 19, 2025
Merged

[O2B-1461] Export refactor#1947
xsalonx merged 35 commits into
mainfrom
xsalonx/export/O2B-1293/export

Conversation

@xsalonx

@xsalonx xsalonx commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • NA

Notable changes for developers:

  • Refactor export functionality. Add DataExportModel responsible for menaging export
  • Add loadAll method to OverviewPageModel, which fetches all possible data

Changes made to the database:

  • NA

@xsalonx xsalonx self-assigned this Jul 30, 2025
@codecov

codecov Bot commented Jul 30, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 130 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.85%. Comparing base (a44f718) to head (6733500).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/public/models/DataExportModel.js 0.00% 55 Missing ⚠️
...ponents/common/dataExport/exportTriggerAndModal.js 0.00% 36 Missing ⚠️
lib/public/models/OverviewModel.js 0.00% 28 Missing ⚠️
...ib/public/views/Runs/Overview/RunsOverviewModel.js 0.00% 9 Missing ⚠️
lib/public/domain/enums/DataExportTypes.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1947      +/-   ##
==========================================
+ Coverage   45.74%   45.85%   +0.10%     
==========================================
  Files        1016     1018       +2     
  Lines       16796    16825      +29     
  Branches     3038     3034       -4     
==========================================
+ Hits         7684     7715      +31     
+ Misses       9112     9110       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xsalonx xsalonx marked this pull request as ready for review July 31, 2025 09:27
@xsalonx xsalonx requested a review from graduta as a code owner July 31, 2025 09:27
@xsalonx xsalonx changed the title [O2B-1293] Export refactor [O2B-1461] Export refactor Aug 4, 2025
Comment thread lib/public/models/DataExportModel.js Outdated
Comment thread lib/public/models/DataExportModel.js Outdated
Comment thread lib/public/models/DataExportModel.js
Comment thread lib/public/models/DataExportModel.js
Comment thread lib/public/models/DataExportModel.js
Comment thread lib/public/components/common/dataExport/exportTriggerAndModal.js Outdated
Comment thread lib/public/components/common/dataExport/exportTriggerAndModal.js Outdated
Comment thread lib/public/components/common/dataExport/exportTriggerAndModal.js Outdated
Comment thread lib/public/components/common/dataExport/exportTriggerAndModal.js
Comment thread lib/public/views/Runs/Overview/RunsOverviewModel.js
Comment thread lib/public/models/OverviewModel.js Outdated
Bumps [joi](https://github.com/hapijs/joi) from 17.13.1 to 18.0.0.
- [Commits](hapijs/joi@v17.13.1...v18.0.0)

---
updated-dependencies:
- dependency-name: joi
  dependency-version: 18.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@xsalonx xsalonx requested a review from graduta August 12, 2025 13:55
@xsalonx xsalonx merged commit 1ebeee9 into main Aug 19, 2025
23 of 24 checks passed
@xsalonx xsalonx deleted the xsalonx/export/O2B-1293/export branch August 19, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants